Skip to content

Preview of DetectionResult refactoring #255

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
sovrasov wants to merge 3 commits into from
Closed

Preview of DetectionResult refactoring #255

sovrasov wants to merge 3 commits into from

Conversation

@sovrasov
Copy link
Member

@sovrasov sovrasov commented Jan 16, 2025

What does this PR do?

Detection result object is now aligned with Python, also that representation allows memory forwarding.

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@github-actions github-actions bot added the build Related to build scripts label Jan 16, 2025
ashwinvaidya17
ashwinvaidya17 previously approved these changes Jan 17, 2025
View reviewed changes
Copy link
Contributor

@ashwinvaidya17 ashwinvaidya17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@sovrasov sovrasov closed this Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ashwinvaidya17 ashwinvaidya17 ashwinvaidya17 left review comments

Assignees

No one assigned

Labels

build Related to build scripts

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sovrasov @ashwinvaidya17